Adding Dapr Spring Boot Starter.#457
Conversation
| */ | ||
| @Configuration | ||
| @ConditionalOnWebApplication | ||
| @ComponentScan("io.dapr.springboot") |
There was a problem hiding this comment.
Is this the only package to be scanned?
There was a problem hiding this comment.
Yes, it's the package where the Spring beans DaprBeanPostProcessor and DaprController locate, they are the beans provided by dapr to be scanned by spring boot.
| <module>sdk-actors</module> | ||
| <module>sdk-springboot</module> | ||
| <module>examples</module> | ||
| <module>dapr-spring-boot-starter</module> |
There was a problem hiding this comment.
sdk-spring-boot-starter
artursouza
left a comment
artursouza
left a comment
There was a problem hiding this comment.
Please, modify at least one example showcasing how this can be used.
After that we will go into more details on how to get this PR merged, specially if we need to publish a new maven package.
|
@darren-wang Thanks for your contribution. Please, see comments above. This has potential for the springboot community. It would be great to see an example on how. Maybe change one of our examples in this repo demonstrating that. |
…wang/java-sdk into dapr-spring-boot-starter � Conflicts: � sdk-spring-boot-starter/pom.xml � sdk-spring-boot-starter/src/main/java/io/dapr/springboot/DaprAutoConfiguration.java � sdk-spring-boot-starter/src/main/resources/META-INF/spring.factories
|
@artursouza |
Codecov Report
@@ Coverage Diff @@
## master #457 +/- ##
============================================
- Coverage 82.92% 82.77% -0.15%
+ Complexity 931 930 -1
============================================
Files 79 79
Lines 2717 2717
Branches 269 269
============================================
- Hits 2253 2249 -4
- Misses 331 334 +3
- Partials 133 134 +1
Continue to review full report at Codecov.
|
|
Should be replaced by #459 |
2 participants
Description
Adding a spring boot starter for dapr to better integrate with Java spring boot applicaitons.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #441
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: